Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removal of SecureBoot options from ConfApp #432

Merged
merged 2 commits into from
Feb 15, 2025

Conversation

apop5
Copy link
Contributor

@apop5 apop5 commented Jan 14, 2025

Description

ConfApp's secure boot menu allows the user the illusion of being able to configure the setup boot options.

The reality is that secure boot is configured by the system prior to ConfApp, and that changes made in ConfApp are not honored on the next boot.

Removing the Secure Boot menu, associated tests and reduce the libraries needed.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

CI Build.
Boot on Qemu to verify functionality.

Integration Instructions

No Changes required.

@apop5 apop5 requested a review from kuqin12 January 14, 2025 21:24
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Jan 14, 2025
@apop5 apop5 requested a review from os-d January 14, 2025 21:24
@github-actions github-actions bot added the impact:breaking-change Requires integration attention label Jan 14, 2025
@apop5 apop5 removed the impact:non-functional Does not have a functional impact label Jan 15, 2025
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Feb 14, 2025
@apop5 apop5 merged commit 646bbcb into microsoft:main Feb 15, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:breaking-change Requires integration attention impact:non-functional Does not have a functional impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants